-
Notifications
You must be signed in to change notification settings - Fork 67
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(manager): block gossiping misbehavior detection #1117
base: srene/fix-test-p2p-validation
Are you sure you want to change the base?
feat(manager): block gossiping misbehavior detection #1117
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1117 +/- ##
==========================================
- Coverage 19.54% 18.63% -0.91%
==========================================
Files 165 168 +3
Lines 41268 44785 +3517
==========================================
+ Hits 8065 8346 +281
- Misses 32034 35232 +3198
- Partials 1169 1207 +38 ☔ View full report in Codecov by Sentry. |
debc18c
to
fcd2eb0
Compare
uerrors.ErrGroupGoLog(eg, m.logger, func() error { | ||
return m.ValidateLoop(ctx) | ||
}) | ||
go uevent.MustSubscribe(ctx, m.Pubsub, "syncLoop", settlement.EventQueryNewSettlementBatchAccepted, m.onNewStateUpdate, m.logger) |
Check notice
Code scanning / CodeQL
Spawning a Go routine Note
return m.ValidateLoop(ctx) | ||
}) | ||
go uevent.MustSubscribe(ctx, m.Pubsub, "syncLoop", settlement.EventQueryNewSettlementBatchAccepted, m.onNewStateUpdate, m.logger) | ||
go uevent.MustSubscribe(ctx, m.Pubsub, "validateLoop", settlement.EventQueryNewSettlementBatchFinalized, m.onNewStateUpdateFinalized, m.logger) |
Check notice
Code scanning / CodeQL
Spawning a Go routine Note
7e8a12a
to
54d729b
Compare
PR Standards
ADR: Optimistic Block Gossiping Misbehavior - Part 1
ADR: Optimistic Block Gossiping Misbehavior - Part 2
Opening a pull request should be able to meet the following requirements
--
PR naming convention: https://hackmd.io/@nZpxHZ0CT7O5ngTp0TP9mg/HJP_jrm7A
Close #1109 #1116
<-- Briefly describe the content of this pull request -->
For Author:
godoc
commentsFor Reviewer:
After reviewer approval: